Skip to content

Fixing the \n problem#3

Open
amstramgram0 wants to merge 31 commits intoperror:masterfrom
amstramgram0:master
Open

Fixing the \n problem#3
amstramgram0 wants to merge 31 commits intoperror:masterfrom
amstramgram0:master

Conversation

@amstramgram0
Copy link

No description provided.

mrglm and others added 4 commits January 14, 2020 20:54
tracker now expects a text file with on each line the executable's name
and arguments, allowing it to produces multiple traces
Copy link
Owner

@perror perror left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, this is a full reindentation of the code... Difficult to see what you did change or not... I do not think I will merge this without any visibility of what really change. Yet, I think I'll take your trace_t API.

/* Free every element in the trace t */
void trace_delete (trace_t *t);

/* Returns a pointer to the first element where t2 differs from t1 */
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It should be mentioned that it returns NULL if both traces are equals.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants